Skip to content

Fix AppEvents startup inbox readiness gating - #247

Merged
drmoisan merged 1 commit into
mainfrom
bug/appevents-loadasync-inbox-gating-243
Jul 6, 2026
Merged

Fix AppEvents startup inbox readiness gating#247
drmoisan merged 1 commit into
mainfrom
bug/appevents-loadasync-inbox-gating-243

Conversation

@drmoisan

@drmoisan drmoisan commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Gates startup inbox processing in AppEvents.LoadAsync() so the hooked-events path waits for the readiness hookup that populates OlInboxes.
  • Runs startup inbox processing from the readiness hookup path after inbox subscriptions are established.
  • Adds focused regression coverage for pre-readiness suppression and post-hookup processing order.

Verification

  • dotnet tool run csharpier format .
  • Analyzer build passed with 0 errors.
  • Nullable warnings-as-errors build passed with 0 warnings and 0 errors.
  • Focused affected tests passed: 16/16.
  • Full TaskMaster.Test passed: 198/198.
  • Full coverage run passed: 4,972/4,972 tests.
  • Corrected repository coverage: 79.9920%; baseline: 79.9234%; changed executable line coverage: 100.0000%.

Exception

  • Repository-wide coverage is 0.0080 percentage points below the 80.0000% threshold.
  • This threshold exception was explicitly authorized and is recorded in docs/features/active/2026-07-06-appevents-loadasync-inbox-gating-243/runbooks/coverage-threshold-exception.runbook.md.

Closes #243

Move startup inbox processing for hooked-event startup into the readiness hookup path after OlInboxes are populated. Add focused regression coverage for the pre-readiness gap and the post-hookup processing order. Record issue #243 orchestration evidence, including the authorized coverage-threshold exception.
@drmoisan
drmoisan force-pushed the bug/appevents-loadasync-inbox-gating-243 branch from dbee996 to e381177 Compare July 6, 2026 22:02
@drmoisan
drmoisan merged commit a8bbd30 into main Jul 6, 2026
2 checks passed
@drmoisan
drmoisan deleted the bug/appevents-loadasync-inbox-gating-243 branch July 18, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: appevents-loadasync-inbox-gating

1 participant